Skip to content

Reduce unnecessary Modal CI runs on PRs#661

Merged
MaxGhenis merged 3 commits into
mainfrom
codex/cheaper-pr-ci
Mar 29, 2026
Merged

Reduce unnecessary Modal CI runs on PRs#661
MaxGhenis merged 3 commits into
mainfrom
codex/cheaper-pr-ci

Conversation

@MaxGhenis
Copy link
Copy Markdown
Contributor

Summary

  • add PR workflow concurrency so stale runs are canceled automatically
  • decide between full Modal CI and runner-only pytest based on a full-data-ci label or critical data-pipeline paths
  • make sure PR CI also triggers for modal_app/**, pyproject.toml, and uv.lock changes

Validation

  • python3 - <<'PY' ... yaml.safe_load(...) ... PY
  • git diff --check

@MaxGhenis MaxGhenis merged commit bc1a996 into main Mar 29, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant